Switch to Github Actions on master (#321)

* Fix build errors and remove clang patch on master branch.

For in-tree build, opencl-clang depends on llvm-project which is
frequently updated everyday, so it is not necessary to have patch
mechanism for clang which will result in many errors when apply patches.

Signed-off-by: haonanya <[email protected]>

* Switch to Github Actions on master

Signed-off-by: haonanya <[email protected]>

* Removed -dwarf-column-info flag and apply suggestions

Signed-off-by: haonanya <[email protected]>

Author: haonanya1

.github/workflows/check-in-tree-build.yml

@@ -0,0 +1,68 @@
+name: In-tree build
+
+on:
+  push:
+    branches:
+      - master
+    paths-ignore: # no need to check build for:
+      - 'docs/**' # documentation
+      - '**.md'   # README
+  pull_request:
+    branches:
+      - master
+    paths-ignore: # no need to check build for:
+      - 'docs/**' # documentation
+      - '**.md'   # README
+  schedule:
+    # Weekly build on Mondays
+    # Ideally, we might want to simplify our regular nightly build as we
+    # probably don't need every configuration to be built every day: most of
+    # them are only necessary in pre-commits to avoid breakages
+    - cron: 0 0 * * 0
+
+env:
+  LLVM_VERSION: 15 
+
+jobs:
+  build_and_test_linux:
+    name: Linux
+    strategy:
+      matrix:
+        build_type: [Release]
+        include:
+          - build_type: Release
+      fail-fast: false
+    runs-on: ubuntu-18.04
+    steps:
+      - name: Checkout LLVM sources
+        uses: actions/checkout@v2
+        with:
+          repository: llvm/llvm-project
+          ref: main 
+          path: llvm-project
+      - name:  Checkout the translator sources
+        uses: actions/checkout@v2
+        with:
+          repository: KhronosGroup/SPIRV-LLVM-Translator
+          ref: master
+          path: llvm-project/SPIRV-LLVM-Translator
+      - name:  Checkout opencl-clang sources
+        uses: actions/checkout@v2
+        with:
+          path: llvm-project/opencl-clang
+      - name: Configure
+        run: |
+          mkdir build && cd build
+          cmake ${{ github.workspace }}/llvm-project/llvm \
+            -DLLVM_ENABLE_PROJECTS="clang" \
+            -DLLVM_EXTERNAL_PROJECTS="llvm-spirv;opencl-clang" \
+            -DLLVM_EXTERNAL_LLVM_SPIRV_SOURCE_DIR=${{ github.workspace }}/llvm-project/SPIRV-LLVM-Translator \
+            -DLLVM_EXTERNAL_OPENCL_CLANG_SOURCE_DIR=${{ github.workspace }}/llvm-project/opencl-clang \
+            -DCMAKE_BUILD_TYPE=${{ matrix.build_type }} \
+            -DLLVM_TARGETS_TO_BUILD="X86" \
+            -G "Unix Makefiles"
+      - name: Build
+        run: |
+          cd build
+          
+          make opencl-clang -j2

options.cpp

@@ -89,7 +89,7 @@ OpenCLArgList *OpenCLOptTable::ParseArgs(const char *szOptions,
     }
 
     unsigned prev = index;
-    Arg *pArg = ParseOneArg(*pArgs, index);
+    std::unique_ptr<Arg> pArg = ParseOneArg(*pArgs, index);
     assert(index > prev && "Parser failed to consume argument.");
 
     // Check for missing argument error.
@@ -101,8 +101,8 @@ OpenCLArgList *OpenCLOptTable::ParseArgs(const char *szOptions,
       break;
     }
 
-    m_synthesizedArgs.emplace_back(std::unique_ptr<llvm::opt::Arg>(pArg));
-    pArgs->append(pArg);
+    pArgs->append(pArg.get());
+    m_synthesizedArgs.emplace_back(std::move(pArg));
   }
   return pArgs.release();
 }

options_compile.cpp

@@ -178,7 +178,6 @@ std::string EffectiveOptionsFilter::processOptions(const OpenCLArgList &args,
   effectiveArgs.push_back("-cl-kernel-arg-info");
   effectiveArgs.push_back("-fno-validate-pch");
   effectiveArgs.push_back("-fno-caret-diagnostics");
-  effectiveArgs.push_back("-dwarf-column-info");
 
   if (std::find_if(effectiveArgs.begin(), effectiveArgs.end(),
                    [](const ArgsVector::value_type& a) {