[PATCH] D103108: [CUDA][HIP] Promote const variables to constant

Fri Jun 4 15:55:56 PDT 2021

tra added inline comments.

================
Comment at: clang/test/CodeGenCUDA/device-use-host-var.cu:68
+// NEG-NOT: @_ZL13var_host_only
+// NEG-NOT: external

----------------
yaxunl wrote:
> sbc100 wrote:
> > This seems to break if the pathname where the llvm checkout lives contains the word "external"
> > 
> > https://logs.chromium.org/logs/emscripten-releases/buildbucket/cr-buildbucket.appspot.com/8845343598940893760/+/steps/LLVM_regression/0/stdout
> > 
> > ```
> > --
> > /b/s/w/ir/cache/builder/emscripten-releases/llvm-project/clang/test/CodeGenCUDA/device-use-host-var.cu:68:13: error: NEG-NOT: excluded string found in input
> > // NEG-NOT: external
> >             ^
> > <stdin>:69:78: note: found here
> > !1 = !{!"clang version 13.0.0 (/b/s/w/ir/cache/git/chromium.googlesource.com-external-github.com-llvm-llvm--project db3e4faa4d2cadf204e67f42bccd98957496a87a)"}
> >                                                                              ^~~~~~~~
> > ```
> > 
> > Unfortunate choice of directory names I know .. but probably worth fixing.
> thanks. this has been fixed by https://reviews.llvm.org/D103658
> 
This should already be fixed by D103658

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D103108/new/

https://reviews.llvm.org/D103108