[llvm-dev] Full restrict support - status update

Sam Elliott via llvm-dev llvm-dev at lists.llvm.org
Tue Oct 29 10:01:57 PDT 2019



> On 29 Oct 2019, at 4:06 am, Finkel, Hal J. via llvm-dev <llvm-dev at lists.llvm.org> wrote:
> 
> On 10/28/19 8:12 PM, Jeroen Dobbelaere via llvm-dev wrote:
>> 
>> - the 'SingleSource/Regression/C/gcc-c-torture/execute/GCC-C-execute-pr38212.test'  in the 'test-suite' fails:
>> -- the test is wrong as it triggers undefined behavior: it reads and writes the same object through 2 different
>>     restrict pointers which have been declared in the same scope.
> 
> 
> This, we can fix now independent of anything else. We should either fix 
> the test, if there's a reasonable way to fix it, or we should remove it. 
> If you can post a patch with an explanation of the problem, that would 
> be great.

I’m wary of updating the test itself, as someone may re-vendor the GCC C torture suite, undoing those edits. However, we already exclude some tests in the GCC torture suite because they rely on UB. Having inspected pr38212.c, I am happy to add this test to the list of excluded tests with the explanation provided. I’ll commit that change today.

Sam

--
Sam Elliott
Software Developer - LLVM
lowRISC CIC
--



More information about the llvm-dev mailing list