[llvm] r191287 - [msan] Handling of atomic load/store, atomic rmw, cmpxchg.

[Johannes Obermayr]

Am Mittwoch, 25. September 2013, 13:05:41 schrieben Sie:
> Btw, there are similar warnings from other parts of LLVM when building with gcc.
> 
> ../tools/clang/lib/CodeGen/CGBuiltin.cpp: In function
> ‘llvm::VectorType* GetNeonType(clang::CodeGen::CodeGenFunction*,
> clang::NeonTypeFlags, bool)’:
> ../tools/clang/lib/CodeGen/CGBuiltin.cpp:1628:1: warning: control
> reaches end of non-void function [-Wreturn-type]
> 
> Is this not an issue for some reason?

I: Program returns random data in a function
E: llvm-clang no-return-in-nonvoid-function /home/abuild/rpmbuild/BUILD/llvm-clang/lib/CodeGen/CGBuiltin.cpp:1628

I couldn't see it before because my llvm-clang package failed earlier because of failing llvm package ...

Please readd "llvm_unreachable("Invalid NeonTypeFlags element type!");"

> 
> 
> On Wed, Sep 25, 2013 at 1:00 PM, Evgeniy Stepanov
> <eugeni.stepanov at gmail.com> wrote:
> > r191361.
> >
> > On Wed, Sep 25, 2013 at 3:11 AM, Johannes Obermayr
> > <johannesobermayr at gmx.de> wrote:
> >> This commit introduces following build error on openSUSE:
> >>
> >> lib/Transforms/Instrumentation/MemorySanitizer.cpp: In member function 'llvm::AtomicOrdering {anonymous}::MemorySanitizerVisitor::addAcquireOrdering(llvm::AtomicOrdering)':
> >> lib/Transforms/Instrumentation/MemorySanitizer.cpp:937:3: warning: control reaches end of non-void function [-Wreturn-type]
> >> lib/Transforms/Instrumentation/MemorySanitizer.cpp: In member function 'llvm::AtomicOrdering {anonymous}::MemorySanitizerVisitor::addReleaseOrdering(llvm::AtomicOrdering)':
> >> lib/Transforms/Instrumentation/MemorySanitizer.cpp:921:3: warning: control reaches end of non-void function [-Wreturn-type]
> >>
> >> ... testing for serious compiler warnings
> >> I: Program returns random data in a function
> >> E: llvm no-return-in-nonvoid-function /home/abuild/rpmbuild/BUILD/llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp:937, 921
> >>
> >> Please add a return value in default: switch or after switch{}.